apps/webapp/app/env.server.ts (1)

1170-1171: Constrain interval and sample rate for safety (reject NaN/out-of-range).

Bound the sample rate to [0,1] and require a positive interval. This also guards against NaN after coercion.

-    EVENT_LOOP_MONITOR_UTILIZATION_INTERVAL_MS: z.coerce.number().int().default(1000),
-    EVENT_LOOP_MONITOR_UTILIZATION_SAMPLE_RATE: z.coerce.number().default(0.05),
+    EVENT_LOOP_MONITOR_UTILIZATION_INTERVAL_MS: z.coerce.number().int().positive().default(1000),
+    EVENT_LOOP_MONITOR_UTILIZATION_SAMPLE_RATE: z.coerce.number().min(0).max(1).default(0.05),

apps/webapp/app/eventLoopMonitor.server.ts (2)

74-90: Make start/stop idempotent and reflect optional state in the type.

Prevents leaking multiple intervals if enable() is called more than once and clears the handle on disable().

-  let stopEventLoopUtilizationMonitoring: () => void;
+  let stopEventLoopUtilizationMonitoring: (() => void) | undefined;

   return {
     enable: () => {
       console.log("🥸  Initializing event loop monitor");

       hook.enable();
-
-      stopEventLoopUtilizationMonitoring = startEventLoopUtilizationMonitoring();
+      if (!stopEventLoopUtilizationMonitoring) {
+        stopEventLoopUtilizationMonitoring = startEventLoopUtilizationMonitoring();
+      }
     },
     disable: () => {
       console.log("🥸  Disabling event loop monitor");

       hook.disable();
-
-      stopEventLoopUtilizationMonitoring?.();
+      if (stopEventLoopUtilizationMonitoring) {
+        stopEventLoopUtilizationMonitoring();
+        stopEventLoopUtilizationMonitoring = undefined;
+      }
     },
   };

---

94-116: Clamp utilization and unref the interval to avoid keeping the process alive.

Clamping keeps logs sane; unref() lets the process exit cleanly.

   let lastEventLoopUtilization = performance.eventLoopUtilization();

-  const interval = setInterval(() => {
+  const interval = setInterval(() => {
     const currentEventLoopUtilization = performance.eventLoopUtilization();

     const diff = performance.eventLoopUtilization(
       currentEventLoopUtilization,
       lastEventLoopUtilization
     );
-    const utilization = Number.isFinite(diff.utilization) ? diff.utilization : 0;
+    const raw = diff.utilization;
+    const utilization = Number.isFinite(raw) ? Math.min(1, Math.max(0, raw)) : 0;

     if (Math.random() < env.EVENT_LOOP_MONITOR_UTILIZATION_SAMPLE_RATE) {
       logger.info("nodejs.event_loop.utilization", { utilization });
     }

     lastEventLoopUtilization = currentEventLoopUtilization;
   }, env.EVENT_LOOP_MONITOR_UTILIZATION_INTERVAL_MS);
+
+  // Don't keep the process alive solely because of this timer (Node.js environments).
+  if (typeof (interval as any).unref === "function") {
+    (interval as any).unref();
+  }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: CR
PR: triggerdotdev/trigger.dev#0
File: .cursor/rules/webapp.mdc:0-0
Timestamp: 2025-08-29T10:06:49.283Z
Learning: Applies to {apps/webapp/app/**/*.server.{ts,tsx},apps/webapp/app/routes/**/*.ts} : Access environment variables only via the env export from app/env.server.ts; do not reference process.env directly

Learnt from: CR
PR: triggerdotdev/trigger.dev#0
File: .cursor/rules/webapp.mdc:0-0
Timestamp: 2025-08-29T10:06:49.283Z
Learning: Applies to apps/webapp/app/**/*.ts : Modules intended for test consumption under apps/webapp/app/**/*.ts must not read environment variables; accept configuration via options instead